Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SNOW-857631 Handle multistatement query type #864

Merged
merged 1 commit into from
Sep 13, 2023
Merged

SNOW-857631 Handle multistatement query type #864

merged 1 commit into from
Sep 13, 2023
+27 −7

Conversation

sfc-gh-pfus
Copy link
Collaborator

@sfc-gh-pfus sfc-gh-pfus commented Jul 26, 2023 •
edited
Loading

Description

Provided way of handling new multistatement query type.

Checklist

  • Code compiles correctly
  • Run make fmt to fix inconsistent formats
  • Run make lint to get lint errors and fix all of them
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary

@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch 3 times, most recently from a293460 to 1558e17 Compare August 2, 2023 12:58
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch 3 times, most recently from e39de92 to 77af1b8 Compare August 11, 2023 07:28
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch 7 times, most recently from c076721 to c023bb9 Compare August 17, 2023 11:51
@sfc-gh-pfus sfc-gh-pfus marked this pull request as ready for review August 17, 2023 11:52
@sfc-gh-pfus sfc-gh-pfus requested a review from a team as a code owner August 17, 2023 11:52
sfc-gh-dheyman
sfc-gh-dheyman reviewed Aug 17, 2023
View reviewed changes
arrow_test.go Outdated Show resolved Hide resolved
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch from c023bb9 to 0d992d5 Compare August 17, 2023 12:02
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch 2 times, most recently from 6339e6d to c96ade5 Compare August 17, 2023 13:05
@sfc-gh-dheyman sfc-gh-dheyman force-pushed the SNOW-857631-multistatement-query-type branch from c96ade5 to 5306af9 Compare August 17, 2023 13:38
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch from 5306af9 to 2081db7 Compare August 17, 2023 16:32
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch 2 times, most recently from 08763a0 to cda48b8 Compare August 31, 2023 07:07
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch from cda48b8 to b56da6a Compare September 11, 2023 07:49
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-857631-multistatement-query-type branch from b56da6a to bdbc328 Compare September 13, 2023 05:54
@sfc-gh-pfus sfc-gh-pfus merged commit 59aa15b into master Sep 13, 2023
21 checks passed
@sfc-gh-pfus sfc-gh-pfus deleted the SNOW-857631-multistatement-query-type branch September 13, 2023 06:37
@github-actions github-actions bot locked and limited conversation to collaborators Sep 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sfc-gh-dheyman sfc-gh-dheyman sfc-gh-dheyman approved these changes

@sfc-gh-dprzybysz sfc-gh-dprzybysz sfc-gh-dprzybysz approved these changes

@sfc-gh-skumbham sfc-gh-skumbham sfc-gh-skumbham approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sfc-gh-pfus @sfc-gh-skumbham @sfc-gh-dprzybysz @sfc-gh-dheyman